Update keyname tables

This commit is very similar to 8c8853a1f5d29d447c16ba5d92bdb7bb442ee721

We update the keynames.txt file from gdkkeynames.h, and we update
keynames-translate.txt to include all the keysym names that we want
to have translations for. Also strip the XF86 from the translatable
keysym names, since we are returning those names now from
gdk_keyval_name().

keyname-table.h is regenerated from these updated files.

Keep XF86 keysym names working

These names are unfortunately stored in gsettings around
the world, so we can't really stop supporting them.

Add some tests for keysyms

Not very extensive, but we do test that XF86 keysym names
keep working.

Update gdkkeysyms.h header

Regenerate these files from the X11 keysymdef.h file, using
gdkkeysyms-update.pl. The only change is the addition of
GDK_KEY_AudioMicMute.

Merge branch 'places-sidebar'

treeview: Use equal areas in the computation of drag dest row positions

We used to divide the row in thirds vertically, and use the outer thirds for GTK_TREE_VIEW_DROP_BEFORE and AFTER, respectively.

Now we use *fourths*.  This is so that we get equal areas for these:

  GTK_TREE_VIEW_DROP_BEFORE
  GTK_TREE_VIEW_DROP_INTO_OR_BEFORE
  GTK_TREE_VIEW_DROP_INTO_OR_AFTER
  GTK_TREE_VIEW_DROP_AFTER

This makes hovering tree rows much more positive.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Adjust the drag destination row when it is above the currently highlighted row

This makes the feedback accurate and without hysteresis.  Haven't I written this code ten times before?

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Dropping on the feedback row is always possible

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Add a temporary row for feedback when inserting a bookmark is possible

We'll prettify this gradually.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

[l10n] Add Tajik translation

Revert "gdk: Update keynames list from gdkkeysyms.h"

This reverts commit 8c8853a1f5d29d447c16ba5d92bdb7bb442ee721.

gdk: Unvfuncify gdk_display_manager_open_display()

This looks like a pretty stupid patch, but it's only a step towards the
ultimate end goal: Get rid of all the displaymanagers.

Revert "wayland: Cache the discovery connection and reuse if possible"

This reverts commit 524ce7f88e26a498c43729f99fda564f5e222582.

wayland: Get rid of generic macros

They're unused.

broadway: Get rid of generic macros

Use existing API instead.

gdk: Update keynames list from gdkkeysyms.h

grep \#define gdkkeysyms.h | sed "s/#define GDK_KEY_\(.*\) \(.*\)/\2
\1/" | grep -v \#define | sort > keynames.txt
./gen-keyname-table.pl keynames.txt keynames-translate.txt

https://bugzilla.gnome.org/show_bug.cgi?id=698385

x11: Simplify function

Displays get made default displays automatically, so there's no need to
do it in the vfunc.

displaymanager: Emit display-opened directly

Instead of letting every backend do it manually.

quartz: Emit GdkDisplay::opened signal

win32: Emit GdkDisplay::opened signal

display: Add display to displaymanager later

Instead of GdkDisplay::init, only add the display to the display manager
in GdkDisplay::opened. This avoids spurious changes of the default
display in gtk_init() when we're trying to find the one that works and
try to open lots of different ones.

display: Make opened signal have a vfunc

... instead of g_signal_connect()ing in every init function.

gdk: Refactor function

Split out a GdkBackend type and iterate over it.

extract-strings: Deal with multiline strings

When putting out N_(...) macros, we must split multiline strings,
otherwise xgettext complains.

Revert more POTFILES.in breakage

Revert adding .ui files to POTFILES.in

This breaks the build unfortunately. Developers will need to resolve that one.

Updated POTFILES.in

Oops.

Remove "tg" from po/LINGUAS until po-properties is fixed

Commit dcad786ba040413dd10e416d03d0cf7f3df226b6 introduced "tg" but
didn't add po-properties/tg.po, which is necessary in order for the
build to not break.

Therefore I'm removing tg until the file is added.

Updated POTFILES.in

Move the drop feedback code to stub functions

For now it is the old, simple call to gtk_tree_view_set_drag_dest_row().  We'll do something
fancier in the following commits.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Add a DropState enum to the internal state of the sidebar

We want to show a little animation when a DnD operation is happening and the user hovers
between existing bookmarks.  The animation should indicate the user that he's about to
create a new bookmark from the file right there.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

doc: Documented icon-shadow CSS property

The docs say that text-shadow applies a drop shadow to both text
and icons, but icons need to use the icon-shadow property.

https://bugzilla.gnome.org/show_bug.cgi?id=698181

Fix mouse interaction in combo cell renderer

Pass the event on when calling gtk_cell_editable_start_editing.
https://bugzilla.gnome.org/show_bug.cgi?id=504901

wayland: Cache the discovery connection and reuse if possible

This will prevent us opening, closing and reopening the same display
connection in the majority of cases.

https://bugzilla.gnome.org/show_bug.cgi?id=694465

Tajik Translation updated

Tajik translation added (translation is still in progress)

menu binding docs: Link to insert_action_group()

It's pretty useless to make a menu without actions behind it and people
who are using gtk_menu_shell_bind_model() directly are probably not
interested in doing it the GtkApplicationWindow way (so they won't get
the "app" and "win" groups for free).  People are going to need to call
gtk_widget_insert_action_group(), so mention this in the docs to help
them along.

bindings: Fix gtk_binding_entry_skip

This fixes a regression from commit 9a3466832cc73ae29b704547747b8699132c1250.

https://bugzilla.gnome.org/show_bug.cgi?id=695380

gdk: Unvfuncify generic key functions

This makes Wayland and X11 no longer call into XKB and libX11 for these
functions but use GDK's own copy of these functions, just like the
win32, quartz and broadway backends.

gdk: Refactor default key vfuncs

Instead of copying them all over the place, keep a default
implementation around.

gdk: Refactor gdk_keyval_convert_case()

... so it doesn't export the function anymore.

broadway: Delete 2 files doing nothing

A function was doing nothing but calling a function that was in its own
source file doing nothing but calling a function in its own source file
that did nothing.

docs: Move property section docs to correct file

GtkSwitch: moved focus drawing from outside border inside the handle.

Merge branch 'merge-places-sidebar'

This is the new GtkPlacesSidebar widget, which is the sidebar used in
GtkFileChooser to display a list of places/folders that the user may
want to use.  File managers are encouraged to use this widget for
their own global list of folders.

Load the file chooser's widgets from the .ui file

This is Tristan's *excellent* work, minus the old code for the shortcuts bar - that is all done
in GtkPlacesSidebar now.

The UI gets loaded from a Glade resource; most of the old code to create the UI by hand is gone.

There is still code for save_widgets_create(); this needs to be moved into the UI file, but it
is not a big deal.

gtk_file_chooser_default_init() calls a new post_process_ui() that takes care of all the things
that cannot be done directly in Glade.

For now, add GtkPlacesSidebar to the Glade catalog for private widgets

Although GtkPlacesSidebar is a public widget, we can't have that in Glade's general catalog
as this branch is not merged into GTK+ proper yet.  We'll move the widget to Glade's
catalog when this is done.

Add .ui file for GtkFileChooserDefault

Updated Interlingua translation

gdk: Make atoms handled generically

This is another step towards making GdkDisplayManager backend-agnostic.

Most of the backends profit from this as their atom implementations
where generic anyway - x11 needed that to allow multiple X displays and
broadway, quartz and wayland don't have the concept of displays.

The X11 backend still did things, so I only #if 0'd some code but did
not actually update anything.

gdk: Remove unused function

displaymanager: Handle list of displays in base class

This moves the add/remove_display() functions from the subclasses to
GdkDisplay and GdkDisplayManager. It also gets rid of the list_displays
vfunc.

doc: Clarify gtk_widget_queue_draw_region() doc

displaymanager: Move set_default_display vfunc

... to GdkDisplayClass.make_default. It's only implemented by X11
anyway.

displaymanager: Handle the default display

... instead of having every backend do it on their own.

print: disconnect source_changed_handler on printeroptionwidget finalize

Printing a web page without window.print (which still segfault),
that is Ctrl+P is fixed (does not attempt to execute the
source_changed_handler on a printeroptionwidget that has been
destroyed) by disconnecting this handler in the printeroptionwidget
finalize.

https://bugzilla.gnome.org/show_bug.cgi?id=696622

gtk-demo: Fix typo

https://bugzilla.gnome.org/show_bug.cgi?id=675649

gtkprintjob: release the surface and backend before the output io channel

If the GtkPrintJob is freed too early when the surface has been created
but the job hasn't been sent to the printer, it's possible that the
file print backend tries to write to the io chaneel when it is already
closed. This produces runtime critical warnings:

GLib-CRITICAL **: g_io_channel_write_chars: assertion `channel->is_writeable' failed

https://bugzilla.gnome.org/show_bug.cgi?id=685420

gtkprintbackendfile: Fix infinite loop in _cairo_write()

It can happen if the io channel has been closed. In that case
g_io_channel_write_chars() returns early because of a g_return macro
that checks if the io channel is writable. When returning from g_return
macros, the bytes written output parameter is not updated and the error
is not filled, so the error is not detected and the written variable is
used uninitialized. We should check the return value of
g_io_channel_write_chars() to break the loop.

https://bugzilla.gnome.org/show_bug.cgi?id=685419

fix mem leak in gtk_drag_set_icon_surface, destroy pattern

https://bugzilla.gnome.org/show_bug.cgi?id=681446

Precache more atoms

Add a few more entries to the list of precached atoms.

wayland: Don't get the shell surface twice

Noticed by Pu Xingyu,
https://bugzilla.gnome.org/show_bug.cgi?id=697947

Revert "Use natural size to set adjustment ranges"

This reverts commit 666d10ec7676a15b6861b785397563cbd4ef21e6.

This change severely broke any treeviews without horizontal
scrollbars. Basically, ellipsization never kicks in, and instead
the treeview content just extends outside the visible area,
rendering it inaccessible. This broke e.g. the control-center
keyboard shortcuts panel, the gnome-disks device list, etc etc.

csd: Don't use a content_window for o-r windows

This fixes drawing of dnd icons.
https://bugzilla.gnome.org/show_bug.cgi?id=697886

wayland: Don't "probe" for a Wayland compositor if started from compositor

In the case that the client is started directly by the compositor the
WAYLAND_SOCKET environment variable is set containing the fd to use that was
created by a socketpair.

This environment variable is consumed by a call to wl_display_connect so a
second call will not take advantage of it.

https://bugzilla.gnome.org/show_bug.cgi?id=697673

Move all of GtkFileChooserDefault's fields into a private structure

I love Emacs macros, by the way.

Move GtkFileChooserDefault to gtkfilechooserdefault.c

We will move all the fields to a private structure next.

Move GtkFileChooserWidgetPrivate to gtkfilechooserwidget.c

In addition, remove the unused "file_system" field from that
structure.

Move GtkFileChooserDialogPrivate into gtkfilechooserdialog.c

We'll take private structures out of gtkfilechooserprivate.h and move
them to the appropriate place.

Remove unused #define

Don't include gtkfilechooserprivate.h from the file chooser's tests

We will make that file a fully private header; the tests have no
business poking inside the file chooser's internals at this point.

GtkPlacesSidebar: Hold a reference to the sidebar while a D-Bus proxy is being created

The creation is async, and the sidebar could be destroyed while the D-Bus proxy is being created.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Merge branch 'places-sidebar' into master

This lands the GtkPlacesSidebar widget.  It is used in
GtkFileChooserDefault, and it can also be used by third-party
applications.

Revert "GtkFileChooserDefault: Define children with a GtkBuilder template"

Having the changes for composite widget templates makes it impossible
to merge the places-sidebar branch.  So, we will merge that branch,
and *then* apply the changes for composite templates.

This reverts commit bf909f56157311174d5d9089bdec954e7def8873.

xi2: Improve pointer emulation debug reporting

Reporting "true" or "false" is nicer than the value of the flag.

https://bugzilla.gnome.org/show_bug.cgi?id=697795

gdkwindow: Handle updates created by outstanding moves in same update

If gdk_window_flush_outstanding_moves() creates new update area
we handle this directly in the same draw to avoid flashing.

This mainly affects win32 as X11 does its exposes from moves async.
However, its important for win32 since ScrollDC seems to sometimes
invalidate (and not copy) unexected regions.

http://bugzilla.gnome.org/show_bug.cgi?674051

win32: Report ScrollDC update region directly

Rather than set the window update region and repaint this region
when we get a WM_PAINT we just directly add it to the update
region. No need to roundtrip via win32.

This lets us also make sure we do this drawing in the same update
cycle. This seems especially important on Win7, because ScrollDC
seems to act kind of weird there, not using bitblt in areas where
it seemingly could, which makes scrolling look really flashy.

http://bugzilla.gnome.org/show_bug-cgi?id=674051

Make gdk_flush_outstanding_moves "reentrant"

If do_move_region_bits_on_impl causes updates they will propely
be moved by the outstanding moves we have not yet flushed.

Update gtk.symbols

This fixes win32 build

win32: Fix build

Need to take ".exe" into account for file dependencies on win32.

win32: Make build with latest gdkdisplay changes

xi2: Reset scroll valuators on synthesized crossing events

On crossing events resulting from moving windows (eg. workspace switch),
deviceid equals sourceid, so make those reset scroll valuators on all
slave devices to avoid misleading jumps in scroll events

Fixes https://bugzilla.gnome.org/show_bug.cgi?id=690275

csd: Use symbolic icons for window buttons

With symbolic icons, we can recolor the icons as needed, and we
can use icon shadows, etc.

Don't free a list that has not been initialized

wayland: Use more explicit out parameter names

Prevent confusion between positional values and hotspot values.

Signed-off-by: Rob Bradford <rob@linux.intel.com>

Update GtkCssProvider box-shadow documentation

Non-inset box-shadows are no longer ignored by the GTK+ theming engine,
outset shadows are supported.

Signed-off-by: Rob Bradford <rob@linux.intel.com>

cds: enable outset shadows

Adds conditional code paths to GdkCssShadowValue for painting outset
shadows, and allows shadows to be applied in two passes (first outset
then inset). This can be used to draw csd shadows in outer window
borders.

https://bugzilla.gnome.org/show_bug.cgi?id=695998
Signed-off-by: Rob Bradford <rob@linux.intel.com>

wayland: don't leave root window values uninitialised

Under Wayland we don't know the absolute position of the device but there are
some API calls that expect to get an root window position. Previously we were
not assigning any value to these out parameters potentially leaving the values
undefined.

This change returns the current surface relative position of the device.

quartz: Fix call to missing symbol

This patch can be considered a leftover of commit 6f607fc8b4cd ,
replace the no longer existing function with the GDK function
meant to replace it.

This was noticeable on regular clipboard operations in quartz

window: Remember current size

Don't just look at previously remembered sizes, also look at the current
size.
This is useful for cases where the window was resized by the user or WM
and not by the application itself.

https://bugzilla.gnome.org/show_bug.cgi?id=696882

window: Split out a function

We don't want to remember sizes in the not resizable case. Also a
function named "guess_default_size" should not look at previous sizes,
it should guess.

window: Remove an unused function.

Luckily, we don't need to support an implementation of a minimal WM for
linux-fb anymore (which is what this function was added for).

docs: Fix Since tag for frame clock functions

wayland: Clean up gdkwayland.h

Make this an include-only header

window: restore size after hide/show properly

Old code assumed the size was stored in widget.allocation. This is no
longer true as the allocation is cleared upon hide. However, we store
the last configure request, and that one tracks the last size, so we can
just use that number.

Sometimes things are so easy - once you figure them out...

https://bugzilla.gnome.org/show_bug.cgi?id=696882

plug: Don't unregister windows that were never registered

Fixes warnings when trying to unregister the socket window. The socket
window is foreign, so we shouldn't register it in the first place.

https://bugzilla.gnome.org/show_bug.cgi?id=697427

build: Fix srcdir != builddir

Extract strings from .ui files without intltool

GtkVolumeButton: Define children with a GtkBuilder template

GtkFontButton: Define children with a GtkBuilder template